From b43d286987fb16a43976b784f506ca4f5517f080 Mon Sep 17 00:00:00 2001
From: Alex Crichton <alex@alexcrichton.com>
Date: Fri, 4 Sep 2015 17:30:59 -0700
Subject: [PATCH] Don't document build dependencies

They're not actually relevant to the documentation, so omit them.
---
 src/cargo/ops/cargo_rustc/context.rs |  6 +++++-
 tests/test_cargo_doc.rs              | 31 ++++++++++++++++++++++++++++
 2 files changed, 36 insertions(+), 1 deletion(-)

diff --git a/src/cargo/ops/cargo_rustc/context.rs b/src/cargo/ops/cargo_rustc/context.rs
index 5cfe98359..93ef0357c 100644
--- a/src/cargo/ops/cargo_rustc/context.rs
+++ b/src/cargo/ops/cargo_rustc/context.rs
@@ -8,6 +8,7 @@ use regex::Regex;
 
 use core::{SourceMap, Package, PackageId, PackageSet, Resolve, Target, Profile};
 use core::{TargetKind, LibKind, Profiles, Metadata, Dependency};
+use core::dependency::Kind as DepKind;
 use util::{self, CargoResult, ChainError, internal, Config, profile};
 use util::human;
 
@@ -420,7 +421,10 @@ impl<'a, 'cfg> Context<'a, 'cfg> {
             pkg.dependencies().iter().filter(|d| {
                 d.name() == dep.name()
             }).any(|dep| {
-                dep.is_transitive() && self.dep_platform_activated(dep, kind)
+                match dep.kind() {
+                    DepKind::Normal => self.dep_platform_activated(dep, kind),
+                    _ => false,
+                }
             })
         }).filter_map(|dep| {
             dep.targets().iter().find(|t| t.is_lib()).map(|t| (dep, t))
diff --git a/tests/test_cargo_doc.rs b/tests/test_cargo_doc.rs
index 42c7a9506..6def20ba2 100644
--- a/tests/test_cargo_doc.rs
+++ b/tests/test_cargo_doc.rs
@@ -348,6 +348,37 @@ test!(target_specific_documented {
                 execs().with_status(0));
 });
 
+test!(no_document_build_deps {
+    let p = project("foo")
+        .file("Cargo.toml", r#"
+            [package]
+            name = "foo"
+            version = "0.0.1"
+            authors = []
+
+            [build-dependencies]
+            a = { path = "a" }
+        "#)
+        .file("src/lib.rs", "
+            pub fn foo() {}
+        ")
+        .file("a/Cargo.toml", r#"
+            [package]
+            name = "a"
+            version = "0.0.1"
+            authors = []
+        "#)
+        .file("a/src/lib.rs", "
+            /// ```
+            /// ☃
+            /// ```
+            pub fn foo() {}
+        ");
+
+    assert_that(p.cargo_process("doc"),
+                execs().with_status(0));
+});
+
 test!(doc_release {
     let p = project("foo")
         .file("Cargo.toml", r#"
-- 
2.30.2